Skip to content

feat(reveal.js): add support for fragment indices in line highlighting#13125

Open
mcanouil wants to merge 10 commits intoquarto-dev:mainfrom
mcanouil:feat/code-line-fragment
Open

feat(reveal.js): add support for fragment indices in line highlighting#13125
mcanouil wants to merge 10 commits intoquarto-dev:mainfrom
mcanouil:feat/code-line-fragment

Conversation

@mcanouil
Copy link
Collaborator

@mcanouil mcanouil commented Jul 24, 2025 •
edited
Loading

Introduce support for fragment indices to enhance line highlighting functionality, allowing for more precise control over highlighted code segments in Reveal.js.

This adds a code-line-fragment-indices attribute to offer control over fragment.
When set, the same number of fragment indices as there are steps are required.

I'm not sure of the other changes that this new feature should be coming with.

---
format: revealjs
---

:::: {.fragment fragment-index="3"}
Step 3
:::

:::: {.columns}

:::: {.column width="50%"}
```{.r code-line-numbers="|2|3" code-line-fragment-indices="1,2,4"}
par(mfrow = c(1, 2))
plot(cars)
plot(cars, col = "red")
par(mfrow = c(1, 1))
```
:::

:::: {.column width="50%"}
```{.r code-line-numbers="|2|3" code-line-fragment-indices="1,2,4"}
par(mfrow = c(1, 2))
plot(cars)
plot(cars, col = "red")
par(mfrow = c(1, 1))
```
:::

:::

:::: {.fragment fragment-index="1"}
First step
:::

@mcanouil mcanouil self-assigned this Jul 24, 2025
@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Jul 24, 2025 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@mcanouil mcanouil changed the title feat: add support for fragment indices in line highlighting (revealjs) feat: add support for fragment indices in line highlighting (reveal.js) Jul 24, 2025
@cderv cderv added this to the v1.9 milestone Jul 24, 2025
@cderv
Copy link
Collaborator

cderv commented Jul 24, 2025

Thanks for initiating !

Could be also related to

Also, do we have an issue about this new feature with a discussion or only the Slack question?
Discussing the issue, the design is sometimes easier, and I may have missed. So maybe the one above covers it.

And regarding naming, I think I would like to handle this one too before adding new options

Just listing all this as thoughts for later.

@mcanouil
Copy link
Collaborator Author

mcanouil commented Jul 24, 2025 •
edited
Loading

Thanks for linking. The changes would indeed fix both issues. (I'm not linking them formally at this point to avoid risking closing the issues)

The PR is mostly here to not lose the code.

@mcanouil mcanouil marked this pull request as ready for review January 6, 2026 13:41
@mcanouil mcanouil changed the title feat: add support for fragment indices in line highlighting (reveal.js) feat(reveal.js): add support for fragment indices in line highlighting Jan 6, 2026
@mcanouil
Copy link
Collaborator Author

mcanouil commented Jan 14, 2026

I updated the PR for it to be more complete: changelog, test, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mcanouil mcanouil

Labels

None yet

Projects

None yet

Milestone

v1.9

Development

Successfully merging this pull request may close these issues.

3 participants

@mcanouil @posit-snyk-bot @cderv